Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update in.ua parser + update specs #135

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jarthod
Copy link

@jarthod jarthod commented Oct 12, 2020

Based on the work made by @Vanav in #69, here is a new PR that takes the latest version of the SUNIC WHOIS reply (yes it changed again), simplifies a bit the time parsing code to use parse_time, and update the specs with accurate new values, including time zone.

I also verified this code with a couple other in.ua hostnames I have and it worked fine with them.
Let me know if there's anything that could be done better.

PS: I'm starting to use whois-parser for my service to notify about domain expiration (and maybe more later). So I have a lot of data about domains and TLD for which the parser is working or not (or has outdated definitions) and am ready to provide some (a lot of?) contributions if you are open. I just thought I would ask you first, not to spend too much time on this if you're not interested in them, let me know!

image

Thanks for this great work !
If this gets merged we can close #69

@jarthod
Copy link
Author

jarthod commented Jan 18, 2021

@weppos little notch on this one 🙇‍♂️

Alternatively if you do not have the time to maintain this any more, as I have a good use for it in my service, I would be happy to help you!

@jarthod
Copy link
Author

jarthod commented Oct 11, 2021

@weppos before I start another fork and try to merge most PR in it to get the gem to parse more domains (which means some users will keep having to find the best fork instead of using the main gem and keep rewriting the same fix other people did). Could you please clarify if you have abandoned the project? and if so if you prefer to let the gem like this (in which case I'll publish my fork under another name) or if you accept to pass it along? Thanks.

@jarthod
Copy link
Author

jarthod commented Nov 9, 2021

For the record I ended up starting my fork in which I'll try to merge most PRs and fix outdated definitions (people who need these unmerged improvements can check it).

gem 'whois-parser', github: 'jarthod/whois-parser' # Unofficial but more up-to-date fork, check status at https://github.com/jarthod/whois-parser

Let me know if you want to catch up at some point.

@jarthod jarthod mentioned this pull request Nov 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant